Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Add macro name and line to macro status 'start' event #1476

Merged
merged 2 commits into from
Jan 21, 2021

Conversation

tiagocoutinho
Copy link

I would be useful to have the macro name and the macro line information in the macro status "start" event.

This PR adds that.

@reszelaz reszelaz self-requested a review January 20, 2021 21:33
@reszelaz reszelaz added this to the Jan21 milestone Jan 20, 2021
Test is running twice the same macro object (created only once).
This looks like a copy&paste error. Remove the second run.
Copy link
Collaborator

@reszelaz reszelaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I just fixed a test that was wrongly programmed and failed here.

@reszelaz reszelaz merged commit 597d3b2 into sardana-org:develop Jan 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants